Skip to content

refactor(@angular/cli): optimize MCP tool descriptions for LLM ergonomics and token usage#33211

Merged
clydin merged 1 commit into
angular:mainfrom
clydin:refactor/mcp-tool-descriptions
May 19, 2026
Merged

refactor(@angular/cli): optimize MCP tool descriptions for LLM ergonomics and token usage#33211
clydin merged 1 commit into
angular:mainfrom
clydin:refactor/mcp-tool-descriptions

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented May 18, 2026

Refactor the descriptions of get_best_practices, search_documentation, and onpush_zoneless_migration tools to improve semantic clarity and reduce system prompt token footprints. Remove redundant operational boilerplate, eliminate internal server logic leakage related to version clamping, and consolidate overlapping iterative process instructions.

@clydin clydin added the target: rc This PR is targeted for the next release-candidate label May 18, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request refactors and condenses the tool descriptions for the Best Practices, Documentation Search, and Zoneless Migration tools within the Angular CLI MCP implementation. The updates simplify the purpose, use cases, and operational notes for each tool to be more concise. Review feedback identifies several instances where a comma was used immediately after the abbreviation 'e.g.' in user-facing messages, recommending their removal to align with general stylistic conventions.

Comment thread packages/angular/cli/src/commands/mcp/tools/doc-search.ts
Comment thread packages/angular/cli/src/commands/mcp/tools/doc-search.ts
Comment thread packages/angular/cli/src/commands/mcp/tools/doc-search.ts
@clydin clydin force-pushed the refactor/mcp-tool-descriptions branch 4 times, most recently from b8eee8f to 2ceb4b8 Compare May 18, 2026 15:58
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label May 18, 2026
@clydin clydin requested a review from alan-agius4 May 18, 2026 16:55
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels May 19, 2026
…mics and token usage

Refactor the descriptions of get_best_practices, search_documentation, and onpush_zoneless_migration tools to improve semantic clarity and reduce system prompt token footprints. Remove redundant operational boilerplate, eliminate internal server logic leakage related to version clamping, and consolidate overlapping iterative process instructions.
@clydin clydin force-pushed the refactor/mcp-tool-descriptions branch from 2ceb4b8 to 464b74b Compare May 19, 2026 10:46
@clydin clydin merged commit 9dc21cc into angular:main May 19, 2026
34 checks passed
@clydin
Copy link
Copy Markdown
Member Author

clydin commented May 19, 2026

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the refactor/mcp-tool-descriptions branch May 19, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/cli target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants